Skip to content

Conversation

@triplingual
Copy link
Contributor

Closes #599

"type": "Manifest"
}
});
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was testing this with my demo site: https://iiif.gdmrdigital.com/import_to_viewers/DragDropDestination.html to see if I could get it working. I had to make an update as you set the data type as text/plain which I think is correct but my app didn't use that value so that should be fixed now.

The other issue I came across is that I think dataTransfer only transfers Strings so when you put the JSON in as the data it converts it to a string of object [object] and you loose the data. I think you need to wrap that in JSON.stringify() to ensure the annotation is converted into a String before passing it .

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I was starting with the bits from that Slack conversation, so I'm happy to get updates.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The logo image has disappeared

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the other hand, when I drag from the recipe in progress I get a strange error in the browser tools:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://newspapers.library.wales/iiif/2.0/image/3412218/info.json. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing). Status code: 403.
Even though the manifest in the script in the recipe is https://iiif.io/api/cookbook/recipe/0006-text-language/manifest.json, not the Wales library one.

@glenrobson
Copy link
Member

There maybe more comments comming from the cookbook editors but we met today and are happy for it to go to the TRC. Suggestions from editors:

I think I've fixed the proof of concept content state but would be good to see if its working for you. Maybe we can discuss in the cookbook call hosting it on the cookbook site.

@triplingual triplingual added the meta: ready-for-editors Ready for Cookbook editor review label Dec 12, 2025
@glenrobson
Copy link
Member

Discussed adding a video showing how the drag and drop works.

Mention its usally the IIIF logo that is used as the image.

@triplingual
Copy link
Contributor Author

triplingual commented Dec 12, 2025

  • capitalization
  • Link to dragged manifest in Example
  • Link to GR's proof of concept
  • Remove 0485 from Related
  • Turn cursor into pointer when hovering over IIIF logo
  • Add a note to the resource providers part about the cursor

@triplingual
Copy link
Contributor Author

Re: c6b43e8, actually changed it to grab 🤦🏻‍♂️

@glenrobson glenrobson merged commit c06c83b into master Jan 23, 2026
1 check passed
@glenrobson glenrobson deleted the 0599-drag-and-drop branch January 23, 2026 14:03
@github-project-automation github-project-automation bot moved this from Cookbook group review to Done in Recipe Progress Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: content state meta: ready-for-editors Ready for Cookbook editor review

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Drag and Drop

3 participants